-
Notifications
You must be signed in to change notification settings - Fork 516
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
breaking: pt: remove data stat from model init #3245
Conversation
Restore deepmodeling#3233 with resolved conflicts and conversations.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## devel #3245 +/- ##
==========================================
+ Coverage 75.19% 75.30% +0.10%
==========================================
Files 372 372
Lines 33136 33220 +84
Branches 1608 1608
==========================================
+ Hits 24916 25015 +99
+ Misses 7347 7332 -15
Partials 873 873 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are the data stats of descriptor and fitting tested by UTs?
source/tests/pt/test_stat.py has the UTs to compare the consistence of data stats between tf and pt, on both descriptor and fitting but only on se_a model. |
Signed-off-by: Duo <[email protected]>
for more information, see https://pre-commit.ci
|
||
def init_desc_stat(self, sumr, suma, sumn, sumr2, suma2): | ||
def init_desc_stat(self, sumr, suma, sumn, sumr2, suma2, **kwargs): |
Check warning
Code scanning / CodeQL
Signature mismatch in overriding method Warning
overridden method
|
||
def init_desc_stat(self, sumr, suma, sumn, sumr2, suma2): | ||
def init_desc_stat(self, sumr, suma, sumn, sumr2, suma2, **kwargs): |
Check warning
Code scanning / CodeQL
Signature mismatch in overriding method Warning
overridden method
|
||
def init_desc_stat(self, sumr, suma, sumn, sumr2, suma2): | ||
def init_desc_stat(self, sumr, suma, sumn, sumr2, suma2, **kwargs): |
Check warning
Code scanning / CodeQL
Signature mismatch in overriding method Warning
overridden method
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per discussion, further refactorization will be provided by a new PR.
Restore deepmodeling#3233 with resolved conflicts and conversations. This PR clean up the data stat process from model init. Please note that this code PR is just an initial cleanup and refinement of the data stat, and a more detailed design of the data stat will be completed in the next PR: - independent data stat from dataloader - data stat support for hybrid descriptors --------- Signed-off-by: Duo <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Restore #3233 with resolved conflicts and conversations.
This PR clean up the data stat process from model init.
Please note that this code PR is just an initial cleanup and refinement of the data stat, and a more detailed design of the data stat will be completed in the next PR: